Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure translator comments end in full stop #420

Merged
merged 1 commit into from
Sep 14, 2021

Conversation

jarolrod
Copy link
Member

This is a follow-up to #318 which addresses this nit by addressing it globally.

This ensures that all GUI translator comments end in a full stop. If a comment does not end in a full stop, a translator may think that the rest of the comment is being cut off.

While here, add a colon to the word "see" for any comments touched which point to look at a link.

This ensures that all gui translator comments end in a full stop.
If a comment does not end in a full stop, a translator may think 
that the rest of the comment is being cut off.

While here, add a colon to the word "see" for any comments
touched which point to look at a link.
Copy link
Member

@hebasto hebasto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 5cc783f, I have reviewed the code and it looks OK, I agree it can be merged.

Copy link
Member

@jonatack jonatack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Light ACK.

I'm not sure about appending a full stop to URLs, but I don't know what the translator UX is like.

@hebasto
Copy link
Member

hebasto commented Sep 13, 2021

I'm not sure about appending a full stop to URLs, but I don't know what the translator UX is like.

I think it's ok as the Transifex recognizes links:

Screenshot from 2021-09-13 17-59-37

Copy link
Contributor

@shaavan shaavan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review ACK 5cc783f

The changes made here look good. Also, I checked and couldn't find any translator comment in src/qt that does not end in full stop and is not addressed here.

This is a small but necessary change to maintain the consistency of translators' comments. Great work, @jarolrod!

@hebasto hebasto merged commit 2161a05 into bitcoin-core:master Sep 14, 2021
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Sep 15, 2021
5cc783f qt: ensure translator comments end in full stop (Jarol Rodriguez)

Pull request description:

  This is a follow-up to #318 which addresses this [nit](bitcoin-core/gui#318 (comment)) by addressing it globally.

  This ensures that all GUI translator comments end in a full stop. If a comment does not end in a full stop, a translator may think  that the rest of the comment is being cut off.

  While here, add a colon to the word "see" for any comments touched which point to look at a link.

ACKs for top commit:
  hebasto:
    ACK 5cc783f, I have reviewed the code and it looks OK, I agree it can be merged.
  shaavan:
    Code Review ACK 5cc783f

Tree-SHA512: 67a1d56175c974e0af9b460fa44163f7ce139a7b81cfaf8ed2c0e7fb6d5120957c3135d96010aeb6229689468e36673fe9571b5a8c3e1c07e047aba1bd563444
@bitcoin-core bitcoin-core locked as resolved and limited conversation to collaborators Sep 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants